Skip to content

Added build script for utf8proc for python3.14 with ubi 10.2 - #8726

Merged
rashmi-ibm merged 4 commits into
ppc64le:masterfrom
shivansh-ibm:utf8proc_rt26.1
Sep 10, 2026
Merged

rashmi-ibm merged 4 commits into
ppc64le:masterfrom
shivansh-ibm:utf8proc_rt26.1

Conversation

@shivansh-ibm

Copy link
Copy Markdown
Contributor

Checklist

  • Have you checked and followed all the points mention in the CONTRIBUTING.MD
  • Have you validated script on UBI 9 container
  • Did you run the script(s) on fresh container with set -e option enabled and observe success ?
  • Did you have Legal approvals for patch files ?

Comment thread u/utf8proc/utf8proc_ubi_10.2.sh Outdated
cmake --build . --target install
# cd $WORK_DIR
cd ..
pip3.14 install setuptools

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use python3.14 -m pip install setuptools instead of pip3.14 install setuptools to fix the gha build failure.
https://github.com/ppc64le/build-scripts/actions/runs/33624677436/job/100230254747?pr=8726#:~:text=temp_build_script.sh%3A%20line,utf8proc_ubi_10.2.sh%20v2.6.1

@shivansh-ibm

Copy link
Copy Markdown
Contributor Author

The package build is passing successfully on UBI 10.2. However, during the test phase, the script fails to find the libutf8proc.so file. These files are generated under the utf8proc/lib64 directory during the utf8proc build.

The current pyproject.toml that we download using wget does not include the lib64 directory in the wheel package. As a result, the built wheel don't have libutfproc.so with current pyproject.toml

I have updated the pyproject.toml to include the lib64 directory as part of the wheel package. Once the PR is merged, the test script should work as expected.

Comment thread u/utf8proc/pyproject.toml
#Package data to add bazel in wheel, this is needed as its not a .py file
[tool.setuptools.package-data]
utf8proc = ["lib/*", "include/*"]
utf8proc = ["lib/*", "lib64/*", "include/*"]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please raise a separate PR to add the path first in pyproject.toml? Once that PR is merged, please trigger a GHA build here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah , that is how it should be done but for now , i have locally verified the updated pyproject.toml is working as expected .

Comment thread u/utf8proc/utf8proc_ubi_10.2.sh

@rashmi-ibm rashmi-ibm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rashmi-ibm
rashmi-ibm merged commit dfc7b3c into ppc64le:master Sep 10, 2026
13 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants